Bump pnpm deps to latest within existing semver ranges - #1234
Merged
Conversation
dgee2
force-pushed
the
deps-bulk-minor-patch
branch
from
August 13, 2026 21:45
3cbbb1e to
9ddb9bc
Compare
dgee2
marked this pull request as ready for review
August 14, 2026 20:41
Contributor
There was a problem hiding this comment.
Pull request overview
This PR updates the ui/menu-website frontend dependency set (minor/patch bumps within existing majors) and adjusts workspace configuration to keep the upgraded toolchain working (removing an obsolete Storybook patch, pinning Vue/MSW via pnpm overrides, and cleaning up newly-stale lint/type workarounds).
Changes:
- Bumped Vue/Quasar/Vite/Vitest/Storybook/ESLint/Playwright/Auth0-Vue and related tooling within current major versions; updated the pinned
pnpmversion. - Removed the Storybook addon-vitest patch and introduced pnpm overrides to force single-Vue resolution and pin MSW for msw-storybook-addon compatibility.
- Removed now-unnecessary eslint suppressions and API response type assertions; added a
.claude/launch.jsonentry for browser preview smoke checks.
Reviewed changes
Copilot reviewed 6 out of 7 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| ui/menu-website/src/services/recipe-service.ts | Removes now-stale @tanstack/query/exhaustive-deps suppressions around vue-query hooks. |
| ui/menu-website/src/services/recipe-api.ts | Drops as RecipeDetail assertions now that generated OpenAPI types make return values correctly typed. |
| ui/menu-website/pnpm-workspace.yaml | Removes patchedDependencies and adds overrides for Vue and MSW resolution stability. |
| ui/menu-website/patches/@storybook__addon-vitest@10.3.3.patch | Removes the previously-applied patch that pnpm can no longer apply. |
| ui/menu-website/package.json | Bulk minor/patch dependency bumps and updates the packageManager pnpm pin. |
| .claude/launch.json | Adds a launch target for dev-server browser preview smoke checks. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
dgee2
force-pushed
the
deps-bulk-minor-patch
branch
from
August 14, 2026 20:57
9ddb9bc to
937fb89
Compare
Bulk minor/patch bump across vue, vue-router, auth0-vue, tanstack-query, vite, vitest, storybook, eslint, playwright, and other tooling in ui/menu-website. All bumps stay within their current major version. Also: - Drop the @storybook/addon-vitest patch: the Windows absolute-path bug it worked around is fixed differently upstream in 10.5.7, and pnpm can no longer apply the patch (ERR_PNPM_UNUSED_PATCH). - Add a pnpm override pinning vue to a single resolved version: @auth0/auth0-vue declares vue as a regular (non-peer) dependency, so bumping our own vue left two incompatible vue installs, breaking app.use(createAuth0()) under vue-tsc. - Add a pnpm override pinning msw to 2.12.14: bumping vitest pulled in msw 2.15 transitively via @vitest/mocker, which is incompatible with msw-storybook-addon@2.0.6's use of worker.context. msw and msw-storybook-addon move together once the addon is upgraded to v3. - Remove now-unused @tanstack/query/exhaustive-deps eslint-disable directives, and two now-unnecessary `as RecipeDetail` assertions in recipe-api.ts, that the eslint/typescript-eslint bumps flag as stale. - Add .claude/launch.json so the dev server can be previewed in-browser for e2e smoke checks. Co-authored-by: Claude <noreply@anthropic.com>
dgee2
force-pushed
the
deps-bulk-minor-patch
branch
from
August 14, 2026 21:12
937fb89 to
2ae92fb
Compare
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Bulk minor/patch bump across vue, vue-router, auth0-vue, tanstack-query, vite, vitest, storybook, eslint, playwright, and other tooling in ui/menu-website. All bumps stay within their current major version.
Also:
@tanstack/query/exhaustive-depseslint-disable directives in recipe-service.ts, and twoas RecipeDetailtype assertions in recipe-api.ts.This is the first PR in a stack of dependency upgrades — see the linked stack for the rest.
Verification
Note: the
as RecipeDetailremoval only shows up as necessary oncesrc/generated/open-api/menu-api.tsexists — that file is gitignored, so a local lint run in a fresh worktree without it will not reproduce the error CI sees.🤖 Generated with Claude Code